Skip to content

[Mellanox] Add PDB (Power Distribution Board) definition to SN6810_LD platform.json for both hardware and SimX platforms#27853

Merged
liat-grozovik merged 1 commit into
sonic-net:masterfrom
Junchao-Mellanox:master_RC_ch6s_pdb
Jun 16, 2026
Merged

[Mellanox] Add PDB (Power Distribution Board) definition to SN6810_LD platform.json for both hardware and SimX platforms#27853
liat-grozovik merged 1 commit into
sonic-net:masterfrom
Junchao-Mellanox:master_RC_ch6s_pdb

Conversation

@Junchao-Mellanox

@Junchao-Mellanox Junchao-Mellanox commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

Why I did it

  • SN6810_LD hardware includes a PDB, but platform.json previously had empty psus and no pdbs entry.
  • Without pdbs in platform.json, DeviceDataManager.get_pdb_count() returns 0 and chassis.initialize_pdb() does not instantiate PDB objects, so PDB status/telemetry is unavailable via platform API.
  • Adding PDB metadata keeps SN6810_LD consistent with other LD platforms and unblocks PDB-related monitoring and validation.
Work item tracking
  • Microsoft ADO (number only):

How I did it

  • Updated device/mellanox/x86_64-nvidia_sn6810_ld-r0/platform.json
  • Updated device/mellanox/x86_64-nvidia_sn6810_ld_simx-r0/platform.json
  • Added under chassis:
"pdbs": [
    {
        "name": "PDB 1"
    }
]

How to verify it

  • Build image for x86_64-nvidia_sn6810_ld-r0 / SimX SKU
  • After boot, verify PDB is initialized:
    • show platform pdb status (or equivalent platform CLI)
    • Python: sonic_platform.chassis.Chassis().get_num_pdbs() returns 1
  • Confirm no regression on SimX platform bring-up

Which release branch to backport (provide reason below if selected)

  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

… platform.json for both hardware and SimX platforms

Signed-off-by: Junchao-Mellanox <junchao@nvidia.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@Junchao-Mellanox

Copy link
Copy Markdown
Collaborator Author

/azpw run Azure.sonic-buildimage

@mssonicbld

Copy link
Copy Markdown
Collaborator

⚠️ Notice: /azpw run only runs failed jobs now. If you want to trigger a whole pipline run, please rebase your branch or close and reopen the PR.
💡 Tip: You can also use /azpw retry to retry failed jobs directly.

Retrying failed(or canceled) jobs...

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@liat-grozovik liat-grozovik merged commit add05b6 into sonic-net:master Jun 16, 2026
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants